Skip to content

Forward-merge release/26.08 into main - #1124

Merged
GPUtester merged 1 commit into
mainfrom
release/26.08
Jul 29, 2026
Merged

Forward-merge release/26.08 into main#1124
GPUtester merged 1 commit into
mainfrom
release/26.08

Conversation

@rapids-bot

@rapids-bot rapids-bot Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Forward-merge triggered by push to release/26.08 that creates a PR to keep main up-to-date. If this PR is unable to be immediately merged due to conflicts, it will remain open for the team to manually merge. See forward-merger docs for more info.

…bit indexing) (#1087)

closes #1076

CuPy recently updated `cupyx.scipy.ndimage.label` to support 64-bit indexing when necessary to support large image sizes. See a more detailed description for the changes made in the upstream PR:
cupy/cupy#9859

This MR makes the exact same change to the very similar kernels underlying `cucim.skimage.measure.label`. The reason we don't use CuPy kernels directly for cuCIM's `label` is that the CuPy API is for binary image labeling only (i.e. all foreground pixels are treated the same regardless of intensity). The version of `label` in the scikit-image API also supports a grayscale mode where neighboring pixels are only connected if the grayscale intensity is actually equal. Thus we have modified kernels that support that logic. The indexing concern and solution is the same as in the upstream PR, though.

Authors:
  - Gregory Lee (https://github.com/grlee77)
  - https://github.com/jakirkham

Approvers:
  - https://github.com/jakirkham

URL: #1087
@rapids-bot
rapids-bot Bot requested a review from a team as a code owner July 29, 2026 21:32
@GPUtester
GPUtester merged commit 63a4ce8 into main Jul 29, 2026
@rapids-bot

rapids-bot Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor Author

SUCCESS - forward-merge complete.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants